Skip to content

Conversation

@colinlyguo
Copy link

@colinlyguo colinlyguo commented Aug 12, 2025

If Feynman is active at genesis and base_fee_per_gas is missing, set it to 0 in the genesis header. Part of: scroll-tech/rollup-node#217.

@colinlyguo colinlyguo requested a review from greged93 August 12, 2025 13:09
@codspeed-hq
Copy link

codspeed-hq bot commented Aug 12, 2025

CodSpeed Performance Report

Merging #320 will not alter performance

Comparing feat-support-genesis-file (27bc553) with scroll (74dd0bd)

Summary

✅ 77 untouched benchmarks

greged93
greged93 previously approved these changes Aug 12, 2025
Copy link

@greged93 greged93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just one suggestion to make it simpler, otherwise lgtm!

Copy link

@greged93 greged93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@colinlyguo colinlyguo merged commit 40affdf into scroll Aug 12, 2025
44 checks passed
@colinlyguo colinlyguo deleted the feat-support-genesis-file branch August 12, 2025 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants